Skip to content

Add -pending.txt tests #463

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 1, 2025
Merged

Conversation

jonshea
Copy link
Contributor

@jonshea jonshea commented Mar 27, 2025

  • -pending.txt test files, containing tests labeled with :skip that highlight known parsing issues in Scala syntax.

  • Updated CONTRIBUTING.md with a brief explanation of the -pending.txt test files.

@jonshea jonshea force-pushed the jonshea/pending-tests branch from 7930d9c to 734beb6 Compare March 30, 2025 20:32
* `-pending.txt` test files, containing tests labeled with `:skip`
  that highlight known parsing issues in Scala syntax.

* Updated `CONTRIBUTING.md` with a brief explanation of the
  `-pending.txt` test files.
@jonshea jonshea force-pushed the jonshea/pending-tests branch from 734beb6 to 5e924ef Compare March 30, 2025 20:38
@jonshea
Copy link
Contributor Author

jonshea commented Mar 30, 2025

The CI failure seems to be an intermittent issues with swift-actions/setup-swift that has been around since August 2023. I’m not sure we have an easy fix since swift-actions/setup-swift is a transitive dependency of tree-sitter/parser-test-action.

@susliko
Copy link
Collaborator

susliko commented Mar 31, 2025

@amaanq Hey 👋🏻
Could you please take a look at CI failures related to swift bindings?

Copy link
Collaborator

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@eed3si9n eed3si9n merged commit c118995 into tree-sitter:master Apr 1, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants